Skip to content

fix: #115/할 일 생성 모달 API 연결#122

Merged
jihye5081 merged 2 commits intomainfrom
feat/#112/mydashboard-page
Apr 2, 2025
Merged

fix: #115/할 일 생성 모달 API 연결#122
jihye5081 merged 2 commits intomainfrom
feat/#112/mydashboard-page

Conversation

@hyeonjiroh
Copy link
Owner

#️⃣ Issue Number

#115


📝 요약

Column.tsx

  • AddTaskButtoncolumnId를 전달

AddTaskButton.tsx

  • Column 컴포넌트에서 columnId를 전달 받고, 클릭 시 해당 columnIduseColumnStoresetSelectedColumnId로 저장

cardsApi.ts

  • 요청 시 보내는 헤더에 오타가 있어 "Content-Type": "application/json",로 오타 수정
  • imageUrl의 타입을 string | null로 수정(이미지를 첨부하지 않는 경우도 있음)
  • API 요청 시 imageUrl이 없는 경우에는 해당 필드를 아예 적지 않아야 되기 때문에(imageUrl: null로 요청 시 400 에러) imageUrl이 유효한 문자열일 때만 payload에 추가하는 방식으로 요청 보내도록 함

CreateTaskModal.tsx

  • useColumStoreselectedColumnId를 가져와서 해당 컬럼 id를 요청 보낼 때 사용
  • assigneeid값이 아닌 userId값으로 요청을 보내야 돼서 이 부분 수정
  • imageUrl의 초기값을 null 값으로 변경
  • 요청 보낼 때 imageUrl이 조건부로 빈 문자열을 보내는 부분 제거(null 값을 보내야 이미지가 없다는 의미로 리퀘스트 보내짐)
  • 요청 후 화면에 바로 반영되도록 window.location.reload();(페이지 새로고침)

🛠️ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 변수명 변경 등)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

@vercel
Copy link

vercel bot commented Apr 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2025 3:52am

@github-actions github-actions bot requested review from ARON-Y and jihye5081 April 2, 2025 03:47
@jihye5081 jihye5081 merged commit 7156201 into main Apr 2, 2025
2 checks passed
@jihye5081 jihye5081 deleted the feat/#112/mydashboard-page branch April 2, 2025 06:24
@hyeonjiroh hyeonjiroh changed the title [fix] #115/할 일 생성 모달 API 연결 fix: #115/할 일 생성 모달 API 연결 Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants